Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aggiunta capitolo DDD #227

Merged
merged 13 commits into from
Oct 31, 2024
Merged

Aggiunta capitolo DDD #227

merged 13 commits into from
Oct 31, 2024

Conversation

baldarn
Copy link
Contributor

@baldarn baldarn commented May 10, 2024

Vedi
#226

@baldarn
Copy link
Contributor Author

baldarn commented May 14, 2024

@BrianAtzori @Cadienvan wdyt?

@baldarn baldarn marked this pull request as ready for review May 14, 2024 09:18
Copy link
Member

@BrianAtzori BrianAtzori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per me il contenuto è buono, c'`e solo qualche errorino da sistemare ma a me fila ed è tutto chiaro!
Lascio la parola anche ad altri ambassador che hanno una seniority piu' navigata rispetto alla mia per un giudizio sui contenuti, ma per me hai prodotto un bel testo, bravo!

docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
@Cadienvan
Copy link
Member

Non sono sparito! Ahah sabato prossimo mi sono preso un momento per leggere come si deve il contenuto! Scusatemi ma tra mille impegni e tutti i capitoli in corso devo sdoppiarmi!

@baldarn baldarn requested a review from BrianAtzori May 20, 2024 08:54
Copy link
Member

@Cadienvan Cadienvan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il contenuto mi torna, farei una introduzione più approfondita del contesto in cui spicca il DDD e il perché dovrei considerare l'adozione di questa metodologia nella mia quotidianità. Considerando il target entry-level del libro, ho paura che si perda il "perché", mentre il "cosa" è piuttosto chiaro e ben definito!

BrianAtzori
BrianAtzori previously approved these changes May 29, 2024
Copy link
Member

@BrianAtzori BrianAtzori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mi torna tutto!
Come sempre passo la palla a qualche ambassador più navigato per un'ulteriore conferma ma secondo me ci siamo, ho visto che hai approvato anche i change che ti ho suggerito quindi top, grande! 🙌🏻

@Cadienvan
Copy link
Member

Ciao @baldarn cosa ne pensi del mio suggerimento sull'intro esplicativa?

@baldarn
Copy link
Contributor Author

baldarn commented Jun 26, 2024

@Cadienvan ho scritto una breve intro con pro e contro, se è ok mergiamo, poi arriverà gente che aggiungerà/toglierà cose, io sono solo un dilettante XD

docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
docs/it/domain-driven-design.md Outdated Show resolved Hide resolved
@baldarn baldarn requested a review from BrianAtzori June 27, 2024 08:59
Cadienvan
Cadienvan previously approved these changes Jul 9, 2024
@baldarn baldarn dismissed stale reviews from AngeloAvv and Cadienvan via 444445c August 23, 2024 08:01
Cadienvan
Cadienvan previously approved these changes Sep 5, 2024
guizzo
guizzo previously approved these changes Sep 6, 2024
@Cadienvan Cadienvan linked an issue Sep 7, 2024 that may be closed by this pull request
AngeloAvv
AngeloAvv previously approved these changes Sep 15, 2024
BrianAtzori
BrianAtzori previously approved these changes Sep 16, 2024
@Cadienvan
Copy link
Member

Viste le modifiche ai criteri di approval, 3+1 è ora un numero sufficiente per pubblicare il contenuto! Congratulazioni @baldarn per essere diventato ufficialmente parte di questo progetto e grazie infinite per il tuo supporto, tra pochi minuti il capitolo sarà online!

@Cadienvan Cadienvan dismissed stale reviews from BrianAtzori, AngeloAvv, guizzo, and themself via f9db9d5 October 31, 2024 08:11
@Cadienvan Cadienvan merged commit ada8455 into Il-Libro-Open-Source:main Oct 31, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Pubblicato
Development

Successfully merging this pull request may close these issues.

[🆕]: Domain Driven Design
5 participants